Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updated CONTRIBUTING.md 2.7.f #6974

Merged

Conversation

andyphancode
Copy link
Member

@andyphancode andyphancode commented Jun 8, 2024

Fixes #6185

What changes did you make?

  • Added a statement to section 2.7.f of CONTRIBUTING.md

Why did you make the changes (we will use this info to test)?

Screenshots of Proposed Changes Of The Website (if any, please do not screen shot code changes)

Visuals before changes are applied

image

Visuals after changes are applied

image

Copy link

github-actions bot commented Jun 8, 2024

Want to review this pull request? Take a look at this documentation for a step by step guide!


Note that CONTRIBUTING.md cannot previewed locally; rather it should be previewed at this URL:

https://github.com/andyphancode/website/blob/update-contributing-6185/CONTRIBUTING.md  

@github-actions github-actions bot added role: front end Tasks for front end developers role: back end/devOps Tasks for back-end developers Feature: Wiki Feature: Onboarding/Contributing.md Complexity: Small Take this type of issues after the successful merge of your second good first issue size: 0.5pt Can be done in 3 hours or less labels Jun 8, 2024
@iancooperman
Copy link
Member

Review ETA: End of today

iancooperman
iancooperman previously approved these changes Jun 9, 2024
Copy link
Member

@iancooperman iancooperman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

@t-will-gillis t-will-gillis self-requested a review June 10, 2024 03:23
Copy link
Member

@t-will-gillis t-will-gillis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@andyphancode Great job!

  • branches look good
  • original issue is linked
  • there is a concise description of what changes were made and why
  • there are visuals for the changes, (which might not be needed, but no harm in this case!)

Everything looks great, however, I have a couple minor requests:

  • Would you please remove the extra ticks in "What changes did you make?" and "Why did you make the changes"
  • This is outside of the scope of this issue, but if you agree I feel like git bash should be capitalized for consistency, on the line that begins IMPORTANT: ...

Thanks for working on this

Copy link
Member

@t-will-gillis t-will-gillis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I will make the call. This was shown incorrectly on the original issue, and is an error that needs to be fixed:

Please capitalize "Git Bash" in line 738, or provide an argument why you believe this should not be done.

@andyphancode
Copy link
Member Author

I will make the call. This was shown incorrectly on the original issue, and is an error that needs to be fixed:

Please capitalize "Git Bash" in line 738, or provide an argument why you believe this should not be done.

No disagreement, just believed it was outside the scope of the issue #. Agreed it should be changed. @t-will-gillis

Copy link
Member

@t-will-gillis t-will-gillis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey @andyphancode Great, thanks so much

@t-will-gillis t-will-gillis merged commit 4fdf8ee into hackforla:gh-pages Jun 12, 2024
3 checks passed
@andyphancode andyphancode deleted the update-contributing-6185 branch June 12, 2024 02:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Complexity: Small Take this type of issues after the successful merge of your second good first issue Feature: Onboarding/Contributing.md Feature: Wiki role: back end/devOps Tasks for back-end developers role: front end Tasks for front end developers size: 0.5pt Can be done in 3 hours or less
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update CONTRIBUTING.md Section 2.7.f
3 participants